-
Notifications
You must be signed in to change notification settings - Fork 108
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Prepare release v4.12.0 #591
Conversation
CHANGELOG.md
Outdated
Update Go to 1.23 | ||
Update other go.mod dependencies | ||
Update VolumeSnapshot, VolumeSnapshotContents, and VolumeSnapshotClasses CRDs to pull in CEL validation rules new in external-snapshotter v8 release | ||
Remove VolumeSnapshot and VolumeSnapshotContets from the snapshot validating webhook configuration as those are now validated via CEL rules on their CRDs |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit: typo with VolumeSnapshotContets
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fixed - TY!
## v4.12.0 - 2024.08.16 | ||
|
||
* Update CSI driver for Kubernetes 1.31 | ||
Update CSI sidecars to latest |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
i think it will be cleaner to have them in bullet points *
like done below and also link the relevant PRs 🙂
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
They're all in the same PR though
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oh it's all going in the same paragraph when rendered. Let me find something better
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Added newlines so that in the rendered .md those are kept on separate lines
Still a single PR though
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
Got wondering for a bit if this should be a major release or not, due to the breaking change in exernal-snapshotter we pulled in, but ultimately decided it's not a breaking change for csi-digitalocean
UPD: I'm probably overthinking it. The 4.12.0 version is for do-csi-plugin, the validating webhook configuration is just here so that we can e2e testing and for convenience